Лабораторная работа 1 #2
Conversation
There was a problem hiding this comment.
2.5 из 4 баллов.
Работа выполнена хорошо! По тех. части вопросов нет, мелкие замечания по оформлению. Придерживайтесь рекомендаций PEP8 относительно пробелов. Обычно IDE выделяет их серым цветом. Учитывайте, пожалуйста, их в будущих работах.
Плюс нейминг
Комментарии делятся на следующие категории:
GOOD- хорошее решение. Хорошее или креативное решение.TODO- можно лучше. Эти замечания не влияют на оценку, и носят рекомендательный характер.FIXME- необходимо исправить. Устраните эти замечания, чтобы получить максимальный балл.
После проверки работа, может содержать несколько статусов:
- Changes requested. Следует выполнить доработки, чтобы получить максимальный балл.
Исправление замечаний следует проводить в IDE и приложить согласно инструкции в этот pull request исправленные файлы. Файлы с исправлениями прикладывайте с теми же названиями! - Changes approved. Работа выполнена на максимальный балл.
Рекомендуется учесть комментарии рекомендательного характера, и приложить в этот pull request исправленные файлы. В этом случае повторно на проверку присылать работу не нужно. Можно нажимать зеленую кнопкуMerge pull request.
Лабораторная работа 1/task_1.py
Outdated
There was a problem hiding this comment.
GOOD
Отличное решение завести переменную, а не пользоваться магическими числами! Заведя переменную и заполнив её на первом этапе вручную, можно в будущем доработать так, чтобы эта переменная вычислялась автоматизированно.
Лабораторная работа 1/task_1.py
Outdated
Лабораторная работа 1/task_2.py
Outdated
There was a problem hiding this comment.
GOOD
Прекрасно, что воспользовались константами
Лабораторная работа 1/task_2.py
Outdated
There was a problem hiding this comment.
FIXME
Переменные в python называют в стиле snake_case: floppy_capacity_space, ...
Поправьте названия этих переменных
Лабораторная работа 1/task_2.py
Outdated
There was a problem hiding this comment.
FIXME
Расставьте между умножением пробелы
Придерживайтесь рекомендаций PEP8 относительно пробелов.
Лабораторная работа 1/task_2.py
Outdated
Лабораторная работа 1/task_3.py
Outdated
There was a problem hiding this comment.
FIXME
Вместо того, чтобы делить на 2 и приводить к целому числу воспользуйтесь целочисленным делением на 2.
There was a problem hiding this comment.
GOOD
Отличное решение завести переменную, а не пользоваться магическими числами!
Лабораторная работа 1/task_3.py
Outdated
Лабораторная работа 1/task_4.py
Outdated
There was a problem hiding this comment.
FIXME
поправить нейминг переменной на стиль snake_case
Все предельно просто и понятно
🐍 👍